Coverage remained the same at 93.648% when pulling ccac52d on raviolin:fix-checking-response-body into a142aee on vpulim:master.

I swear it passed all tests on my local machine, but one Travis job failed to npm install for some reason...
https://travis-ci.org/vpulim/node-soap/jobs/299410819

@raviolin sorry for letting you down for this long... I had some time off and lost sight of this issue. Could you please try and rebase against the latest master?

I updated our .travis.yaml file in order to test against the latest LTS instead of the latest available version (which seems to fail the tests) - now it should run as expected.

Coverage remained the same at 93.648% when pulling 21e4d0b on raviolin:fix-checking-response-body into 3db3355 on vpulim:master.

Thanks a lot @raviolin 👍

In order to get your PR pushed I'll have to ask for at least 1 test for your contribution as requested per our Guidelines on Submitting a Pull Request

why check for obj.html anyway? Does this lib ever use obj.html? Seems we could just move the if statement up and call this good.

@jsdevel apparently it's necessary in case the client gets an HTML response. There are tests for that.

@herom I'll add a test with a response sample, please don't close this PR yet.

Coverage remained the same at 93.648% when pulling c7cca26 on raviolin:fix-checking-response-body into 3db3355 on vpulim:master.

Coverage remained the same at 93.587% when pulling 24736cd on raviolin:fix-checking-response-body into 905eadb on vpulim:master.

LGTM

Can you remove 'needs tests' label? Or should I add more tests?

@raviolin I don't know why the label is still there and we haven't merged your PR yet.

Sorry for that... again 🐑 🐑 🐑 😞